-
Notifications
You must be signed in to change notification settings - Fork 735
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[UR][L0] Upgrade L0 loader to v1.15.1 #11844
Conversation
885f972
to
6ad721b
Compare
@smaslov-intel : I need this for several new features in L0 spec. is it ok to upgrade the L0 loader here in the intel/llvm? I'm upgrading it in the UR L0 adapter here oneapi-src/unified-runtime#1061 |
I think we should then also update the L0 loader that is used in CI testing. |
thanks @smaslov-intel . How can we do that? is it something I can add as part of this PR or does it need to be done by someone else? |
@bader : could you help answering this? |
@aelovikov-intel, could you suggest the way to test L0 loader update? |
Sorry, I don't know how the closing happened... |
Why can't we get that update as part of the GPU driver? We don't want to run our testing with pre-release GPU stack... |
It looks we need all the right components spelled (in sync with each other) in https://github.com/intel/llvm/blob/sycl/devops/dependencies.json |
6ad721b
to
79164fa
Compare
thanks @aelovikov-intel, Definitely, we can update the loader as part of the GPU driver. How can we do that? What is the next GPU driver to be upgraded, and when is it planned? |
f23d4ba
to
2103cfa
Compare
Use a new environment variable, UR_L0_LEAKS_DEBUG, to check for leaks in the UR L0 adapter, instead of relying on a specific value being set in UR_L0_DEBUG. Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
This to use latest features present in L0 spec. Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cosmetic modification to graphs tests
@intel/llvm-gatekeepers please merge. |
This to use latest features present in L0 spec. --------- Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com> Co-authored-by: Kenneth Benzie (Benie) <k.benzie@codeplay.com>
This to use latest features present in L0 spec.